-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AccessRight: 'open','embargoed', 'restricted','closed' #352
Conversation
Add accessRight: 'open','embargoed', 'restricted','closed' for Zenodo
Documentation needs to be update ! |
I would be in favor to use an |
For the documentation of the metadata pivot metadata model it is through a CSV file that list possible keys and mark them if they are used in undelying actions: https://github.com/r-geoflow/geoflow/blob/master/doc/Summary_metadata.csv |
Change `embargoDate` in Rights to `embargo` in Date
Done ! Could you please review it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kikislater the property for entity dates is dates
, and not date
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix entity$date
to entity$dates
Sorry for that
@kikislater can you implement a control with proper error log in case embargo date is missing (I assume it's a blocker in Zenodo if we don't specify this date) |
Add control to embargo date
Remove unexpected dash from previous PR #352
Add accessRight and dependant (embargoDate, accessConditions) for Zenodo to geoflow
Tested with following drive shit: https://docs.google.com/spreadsheets/d/1wTyq-2sJgdvpTzE4309Lyo1vOac7uIb1dMeHEfKcqbg/edit?usp=sharing
and following json :
https://github.com/IRDG2OI/geoflow-g2oi/blob/sylvain/Zenodo.json
@juldebar